Add pagination_extra_css to tag bootstrap_pagination#765
Add pagination_extra_css to tag bootstrap_pagination#765plenaerts wants to merge 14 commits intozostera:mainfrom
Conversation
|
I'm wondering if the same effect (without adding to the library) could be obtained by And then styling your CSS using Would that work for you? |
I don't think I agree, no. Applying the class directly to the element is more clear and simple:
So, while there may be many cases where another div "might just work", I think applying the class directly to the pagination ul is the more clear and more clean way. |
|
How about we add support for |
|
I think that would be more consistent with the other django-bootstrap5 elements, yes. Two questions:
|
|
I'd keep it limited to just pagination and use |
|
Great. Thanks for your feedback! I'll rename the parameter when I find the time and let you know. |
|
Parameter is renamed to extra_classes. |
I wanted to tweak the looks of the pagination block, so I added a parameter to the existing tag bootstrap_pagination which attaches an extra CSS class to the rendered pagination ul element.
Docs and tests are included.
Please consider merging, or let me know if you want me to correct or improve things, or even if you find it just useless :-)